Skip to content

refactor(store): Extract foundation-store leaf#1164

Merged
leumor merged 1 commit intodevelopfrom
feature/foundation-store-extraction
Mar 24, 2026
Merged

refactor(store): Extract foundation-store leaf#1164
leumor merged 1 commit intodevelopfrom
feature/foundation-store-extraction

Conversation

@leumor
Copy link
Collaborator

@leumor leumor commented Mar 24, 2026

Summary

  • create a new :foundation-store leaf module and wire the root project to depend on it
  • move the main network.crypta.store, network.crypta.store.caching, and network.crypta.store.saltedhash production sources into the new leaf without changing package names
  • add selective ownership metadata for :foundation-store and claim network/crypta/store/alerts/** in :foundation-store-contracts
  • keep the existing store tests in the root project for this extraction PR

How to test

  • ./gradlew verifySelectiveLeafOwnershipMetadata
  • ./gradlew :foundation-store:compileJava
  • ./gradlew compileJava
  • ./gradlew test --tests "network.crypta.store.*"
  • ./gradlew test --tests "network.crypta.store.caching.*"
  • ./gradlew test --tests "network.crypta.store.saltedhash.*"
  • ./gradlew compileTestJava buildJar

Notes

  • this PR is limited to physical extraction, build wiring, and ownership metadata
  • store tests remain in root intentionally

Move the main network.crypta.store implementation sources into a new
foundation-store leaf without changing package names.

Wire the root project to depend on the new leaf and update selective
ownership metadata for foundation-store and foundation-store-contracts.
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
74.8% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@leumor leumor merged commit 8539691 into develop Mar 24, 2026
7 of 8 checks passed
@leumor leumor deleted the feature/foundation-store-extraction branch March 24, 2026 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant